Skip to content

Conversation

@aritzolea
Copy link

…nd invoice have foreign currencies and fix tests

}
)
test_invoice.action_post()
self.assertAlmostEqual(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤯

@aritzolea aritzolea force-pushed the 16.0-fix-sale_order_invoice_amount branch from 84b2572 to 98be8df Compare December 22, 2025 13:11
…nd invoice have foreign currencies and fix tests
@aritzolea aritzolea force-pushed the 16.0-fix-sale_order_invoice_amount branch from 98be8df to c5c706d Compare December 22, 2025 14:24
@aritzolea aritzolea changed the title [FIX] sale_order_invoice_amount: Fix invoice amount calc when order a… [16.0][FIX] sale_order_invoice_amount: Fix invoice amount calc when order a… Dec 30, 2025
@aritzolea aritzolea marked this pull request as ready for review January 8, 2026 14:46
@aritzolea
Copy link
Author

Hi @rousseldenis could you review and merge this if OK? Thanks in advance

Copy link
Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems legit

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@aritzolea
Copy link
Author

aritzolea commented Jan 13, 2026

Seems legit

Could you merge @rousseldenis ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants